Conversation
✅ Deploy Preview for piech-dev ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
There was a problem hiding this comment.
Pull Request Overview
Updates documentation in README.md to improve clarity and accuracy of project descriptions, particularly around GitHub-based project management functionality.
- Refined section heading for better clarity
- Expanded description of project metadata fetching process
- Clarified terminology around GitHub topics and meta keywords
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
There was a problem hiding this comment.
Pull Request Overview
Copilot reviewed 9 out of 10 changed files in this pull request and generated 2 comments.
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
…>s; cleanup & additions
There was a problem hiding this comment.
Pull Request Overview
Copilot reviewed 9 out of 10 changed files in this pull request and generated 2 comments.
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
There was a problem hiding this comment.
Pull Request Overview
Copilot reviewed 9 out of 10 changed files in this pull request and generated 4 comments.
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
There was a problem hiding this comment.
Pull Request Overview
Copilot reviewed 9 out of 10 changed files in this pull request and generated 2 comments.
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
There was a problem hiding this comment.
Pull Request Overview
Copilot reviewed 9 out of 10 changed files in this pull request and generated 2 comments.
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
| // We intentionally only check the 'master' branch as requested, | ||
| // and do not fallback to 'main' here. |
There was a problem hiding this comment.
The comment mentions checking only the 'master' branch 'as requested', but there's no context in the code about who made this request or why. Consider clarifying the reason for this constraint or referencing where this requirement comes from.
| // We intentionally only check the 'master' branch as requested, | |
| // and do not fallback to 'main' here. | |
| // We intentionally only check the 'master' branch, as per project specification (see issue #42), | |
| // and do not fallback to 'main' here. This is required for compatibility with legacy repositories. |
| knowsAbout: Array.from( | ||
| new Set( | ||
| PROJECTS.flatMap((p) => p.technologies).map( | ||
| (t) => TECHNOLOGIES[t].fullName, | ||
| ), | ||
| ), | ||
| ), |
There was a problem hiding this comment.
This computation runs on every meta function call, which could be inefficient during server-side rendering. Consider moving this to a constant or memoizing the result since PROJECTS and TECHNOLOGIES are static.
1 participant
No description provided.